Skip to content

Added Bicep language support - #109

Merged
Ngwerume merged 1 commit into
masterfrom
feature/add-language-to-codacy-bicep
Sep 1, 2026
Merged

Added Bicep language support#109
Ngwerume merged 1 commit into
masterfrom
feature/add-language-to-codacy-bicep

Conversation

@Ngwerume

@Ngwerume Ngwerume commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

This is adding Bicep language support for checkov

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 duplication

Metric Results
Duplication 0

View in Codacy

AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.

Run reviewer

TIP This summary will be updated as you push new changes.

@codacy-production codacy-production Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

The PR adds Bicep language support by defining the Bicep case object and registering it in the Language API. While the implementation aligns with the core requirements, there is a missing file extension (.bicepparam) that is standard for Bicep parameter files. Additionally, although the Codacy analysis is up to standards, there are no unit tests provided to verify the new language registration or its inclusion in the global registry. These gaps should be addressed to ensure robust language detection.

About this PR

  • The PR lacks unit or integration tests to verify the new language registration in Language.all. Additionally, the PR description mentions Checkov support; please ensure no further Bicep-specific configurations or tool-specific file patterns are required beyond this generic registration.

Test suggestions

  • Verify that the Bicep object is correctly instantiated with the .bicep extension.
  • Verify that the Bicep language is included in the comprehensive list of languages (Language.all).
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Verify that the Bicep object is correctly instantiated with the .bicep extension.
2. Verify that the Bicep language is included in the comprehensive list of languages (Language.all).

TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback


case object Terraform extends Language(extensions = Set(".tf"))

case object Bicep extends Language(extensions = Set(".bicep"))

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 MEDIUM RISK

Suggestion: Bicep parameter files use the .bicepparam extension. Adding it to the extensions set ensures these files are correctly identified.

Suggested change
case object Bicep extends Language(extensions = Set(".bicep"))
case object Bicep extends Language(extensions = Set(".bicep", ".bicepparam"))

@Ngwerume
Ngwerume merged commit 78bc5a2 into master Sep 1, 2026
3 checks passed
@Ngwerume
Ngwerume deleted the feature/add-language-to-codacy-bicep branch September 1, 2026 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants